Pull Request Review
/kawasima/プルリクレビュー
観点
様々なレビューの種類があるが、基本的にはレビューは品質を担保するためのプロセス
品質は誰かにとっての価値である
Google の Change Log Review (CL Review) ガイドより https://google.github.io/eng-practices/review/reviewer/looking-for.html
設計 Design
コードはよくデザインされていて、システムにふさわしいか
機能性 Functionality
意図したとおりに動くか
ユーザーにとってよい振る舞いをするコードか
複雑性 Complexity
コードはよりシンプルにできるか
将来そのコードに出くわした開発者は用意に理解できるか
テスト Tests
正しく、そしてよくデザインされた自動テストがあるか
名前重要 Naming
簡潔で明瞭な名前を変数・クラス・メソッドetc.に選んでいるか
コメント Comments
コメントは簡潔で役に立つものか
スタイル Style
スタイルガイドに従っているか
ドキュメント Documentation
関連するドキュメントをアップデートしているか